Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CDPT-2295 Update documentation #68

Merged
merged 11 commits into from
Feb 7, 2025
Merged

Conversation

EarthlingDavey
Copy link
Contributor

@EarthlingDavey EarthlingDavey commented Feb 3, 2025

Hey @EmilyHazlehurst would it be possible for your review to include getting this up and running locally? ... To ensure that I've not missed any steps from the docs.

Copy link

@EmilyHazlehurst EmilyHazlehurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@EarthlingDavey I've added a couple of observations I made whilst setting this up locally.

I've been able to get it to the point of logging the snapshots and successfully authenticating between the local intranet and archiver instances, but I can't see the snapshots as I get a 404 when redirected to the /env-agency/index.html page. I think it has something to do with the proxy as it doesn't seem to get hit, but I've not had chance to investigate it any more than that.

It works well on dev though, so might be something to do with my local setup.

.github/README.md Show resolved Hide resolved
.github/README.md Show resolved Hide resolved
.github/README.md Show resolved Hide resolved
.github/README.md Show resolved Hide resolved
conf/node/server.js Show resolved Hide resolved
@EarthlingDavey
Copy link
Contributor Author

Thanks @EmilyHazlehurst for such detailed feedback 🙌

It's exactly what I need as I probably got quite blind to the requirements and running order.

I'll address and ask for a re-review later :)

@EarthlingDavey EarthlingDavey merged commit 51243fa into main Feb 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants